Skip to content

feat(stovepipe): Pass terminal build to resolver - #757

Merged
prathshenoy merged 1 commit into
prath.shenoy/stovepipe-terminal-build-historyfrom
prath.shenoy/stovepipe-project-result-terminal-build
Sep 30, 2026
Merged

prathshenoy merged 1 commit into
prath.shenoy/stovepipe-terminal-build-historyfrom
prath.shenoy/stovepipe-project-result-terminal-build

Conversation

@prathshenoy

@prathshenoy prathshenoy commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

What:

  • Provide the terminal build ID when resolving project validation results.

Why:

  • Let repository-specific resolution use artifacts from the build that established the request state.

Test Plan

  • Add unit tests.

Revert Plan

  • Revert this PR. It only stops project resolution from receiving the terminal build ID; request state and history remain unchanged.

Issues

Stack

  1. feat(stovepipe): Store terminal build ID #752
  2. feat(stovepipe): Persist terminal build ID #754
  3. feat(stovepipe): Record terminal build history #756
  4. @ feat(stovepipe): Pass terminal build to resolver #757
  5. docs(stovepipe): Document terminal build IDs #758

@prathshenoy prathshenoy changed the title feat(stovepipe): Pass terminal builds to project resolution feat(stovepipe): Pass build to project resolver Sep 29, 2026
@prathshenoy
prathshenoy added this pull request to stack #755 September 29, 2026 01:45
@prathshenoy
prathshenoy marked this pull request as ready for review September 29, 2026 13:35
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-terminal-build branch from 485d573 to ec745fb Compare September 29, 2026 17:18
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-terminal-build branch from ec745fb to 716e4b3 Compare September 29, 2026 17:20
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-terminal-build branch from 716e4b3 to aacd25d Compare September 29, 2026 17:27
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-terminal-build branch from aacd25d to 8d214b3 Compare September 29, 2026 22:22
@prathshenoy prathshenoy changed the title feat(stovepipe): Pass build to project resolver feat(stovepipe): Pass terminal build to resolver Sep 29, 2026
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-terminal-build branch from 8d214b3 to 0bc351f Compare September 29, 2026 22:28
**What**:
- Pass each request’s terminal build identity to project resolution.

**Why**:
- Let project resolution retrieve results from the build that established the request outcome.
@prathshenoy
prathshenoy force-pushed the prath.shenoy/stovepipe-project-result-terminal-build branch from 0bc351f to a4f85b0 Compare September 29, 2026 22:30
@prathshenoy
prathshenoy added this pull request to the merge queue Sep 30, 2026
Merged via the queue into main with commit 1c5b572 Sep 30, 2026
16 checks passed
@behinddwalls
behinddwalls deleted the prath.shenoy/stovepipe-project-result-terminal-build branch September 30, 2026 00:10
behinddwalls pushed a commit to behinddwalls/submitqueue that referenced this pull request Oct 5, 2026
## Summary

**What**:
- Store the build ID that establishes a request's terminal state.

**Why**:
- Make the terminal build direct request state for downstream
resolution.

## Test Plan

- [x] Add unit tests.

## Revert Plan

- Revert this PR. No downstream consumer needs the terminal build
identity currently.

## Issues

-
[CODEM-544](https://linear.app/uber/issue/CODEM-544/pass-terminal-build-to-project-resolver)


## Stack

1. @ uber#752
1. uber#754
1. uber#756
1. uber#757
1. uber#758
behinddwalls pushed a commit to behinddwalls/submitqueue that referenced this pull request Oct 5, 2026
## Summary

**What**:
- Persist the terminal build ID together with a request's terminal
state.

**Why**:
- Ensure the first terminal build also becomes the request’s recorded
terminal build.

## Test Plan

- [x] Add unit tests.

## Revert Plan

- Revert this PR. It only stops terminal transitions from populating the
nullable field; existing request state remains unchanged.

## Issues

-
[CODEM-544](https://linear.app/uber/issue/CODEM-544/pass-terminal-build-to-project-resolver)

## Stack

1. uber#752
1. @ uber#754
1. uber#756
1. uber#757
1. uber#758
behinddwalls pushed a commit to behinddwalls/submitqueue that referenced this pull request Oct 5, 2026
## Summary

**What**:
- Record the terminal build ID with a request's terminal history when a
build establishes the state.

**Why**:
- Preserve durable history showing which build established the request’s
final state.

## Test Plan

- [x] Add unit tests.

## Revert Plan

- Revert this PR. Terminal history is supplementary and does not change
request state or project resolution.

## Issues

-
[CODEM-544](https://linear.app/uber/issue/CODEM-544/pass-terminal-build-to-project-resolver)

## Stack

1. uber#752
1. uber#754
1. @ uber#756
1. uber#757
1. uber#758
behinddwalls pushed a commit to behinddwalls/submitqueue that referenced this pull request Oct 5, 2026
## Summary

**What**:
- Document how a terminal build ID is stored on a request and provided
to project resolution.

**Why**:
- Keep the RFC aligned with the request-side terminal-build behavior.

## Test Plan

- [x] Ask an agent to find inconsistencies between the docs and the
code.

## Revert Plan

- Revert this PR. The changes in this PR include documentation only.

## Issues

-
[CODEM-544](https://linear.app/uber/issue/CODEM-544/pass-terminal-build-to-project-resolver)


## Stack

1. uber#752
1. uber#754
1. uber#756
1. uber#757
1. @ uber#758

This branch was previously deployed

1 inactive deployment
stack-rebase — a4f85b0d Deployed Sep 30, 2026 by prathshenoy via Rebase Stack #541
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants